.github/workflows/prod-cd.yml (1)

82-130: 원격 배포 SSH 스크립트의 복원력 개선이 필요합니다.

현재 SSH 스크립트는 여러 단계를 순차적으로 실행하며 set -e로 에러 시 중단하도록 설정되어 있습니다. 그러나 다음과 같은 시나리오에서 개선이 필요합니다:

상황별 개선 사항:

1. docker pull 실패 시 처리

   • 현재: 에러 발생 시 그냥 중단
   • 제안: 기존 이미지로 서비스 유지 또는 명확한 실패 알림

2. docker compose up -d 실패 시 처리

   • Line 123에서 새 컨테이너 시작 실패 시, 이전 컨테이너가 이미 down된 상태
   • 제안: 스크린샷 또는 파일로 이전 상태 백업 후 롤백 가능하도록 개선

3. Nginx 설정 적용 실패 시 처리

   • Line 111의 nginx -t 검증 후 reload가 실패하면 서비스 불가
   • 제안: 구 설정 백업 후 검증 실패 시 복구

아래와 같이 개선하는 것을 권장합니다:

# Nginx 설정 변경 전 백업
cp /etc/nginx/conf.d/default.conf /etc/nginx/conf.d/default.conf.backup

# 새 설정 적용
sudo cp ./nginx/default.conf /etc/nginx/conf.d/default.conf

# 검증 실패 시 복구
if ! sudo nginx -t; then
  echo "Nginx configuration test failed. Reverting..."
  sudo cp /etc/nginx/conf.d/default.conf.backup /etc/nginx/conf.d/default.conf
  sudo nginx -t
  exit 1
fi

sudo nginx -s reload

또한 Docker Compose도 이전 상태 보존을 고려하세요:

# 이전 상태 백업
docker compose -f docker-compose.prod.yml cp solid-connection-server:/var/log/spring ./logs.backup

# 새 버전으로 재시작
OWNER_LOWERCASE=$OWNER_LOWERCASE IMAGE_TAG=$IMAGE_TAG_ONLY docker compose -f docker-compose.prod.yml up -d

# 헬스 체크 (옵션)
sleep 5
curl -f http://localhost:8080/health || { echo "Health check failed"; exit 1; }

.github/workflows/dev-cd.yml (1)

118-119: Docker Compose 정리 명령어가 prod 워크플로우와 불일치합니다.

현재 dev-cd.yml:

• Line 118: docker compose -f docker-compose.dev.yml down

prod-cd.yml (더 견고함):

• Line 117-120:
  • docker compose ... stop (graceful shutdown)
  • docker compose ... down --remove-orphans (고아 네트워크/볼륨 정리)

차이점:

1. prod는 명시적으로 stop → down 순서 진행
2. prod는 --remove-orphans 플래그 사용 (고아 리소스 제거)
3. dev는 이 두 가지를 누락

이로 인해 dev 환경에서 장기 실행 후 불필요한 네트워크/볼륨이 쌓일 수 있습니다.

prod 워크플로우와 일치하도록 수정하세요:

  # 6. Docker Compose 재시작
  echo "Restarting Docker Compose with tag: $IMAGE_TAG_ONLY"
- docker compose -f docker-compose.dev.yml down
+ echo "Stopping containers gracefully..."
+ docker compose -f docker-compose.dev.yml stop
+ 
+ echo "Removing old containers and networks..."
+ docker compose -f docker-compose.dev.yml down --remove-orphans
+ 
+ echo "Starting new containers..."
  OWNER_LOWERCASE=$OWNER_LOWERCASE IMAGE_TAG=$IMAGE_TAG_ONLY docker compose -f docker-compose.dev.yml up -d

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: whqtker
Repo: solid-connection/solid-connect-server PR: 481
File: docs/infra-config/nginx.conf:36-36
Timestamp: 2025-08-25T10:59:42.734Z
Learning: When using network_mode: host in Docker Compose, containers share the host's network namespace, so nginx proxy_pass should target localhost:PORT instead of container service names like solid-connection-server:PORT.

.github/workflows/prod-cd.yml (2)

40-45: ⚠️ GHCR 로그인 시 소유자명 대소문자 불일치 문제 (이전 리뷰의 미해결 항목)

앞선 리뷰에서 지적되었던 문제가 여전히 남아 있습니다.

문제:

1. Line 44의 GHCR 로그인: ${{ github.repository_owner }} (원본 대소문자 유지)
2. Line 53의 이미지명: ghcr.io/${OWNER_LOWERCASE}/solid-connection-server (소문자 변환)

리포지토리 소유자명에 대문자가 포함되면, 로그인 사용자명과 이미지명의 대소문자 불일치로 인증 오류가 발생합니다.

수정:

  - name: Log in to GitHub Container Registry (GHCR)
    uses: docker/login-action@v3
    with:
      registry: ghcr.io
-     username: ${{ github.repository_owner }}
+     username: ${{ steps.image_meta.outputs.owner_lowercase }}
      password: ${{ secrets.GITHUB_TOKEN }}

그리고 Line 48-54의 메타데이터 정의 단계에 다음 줄을 추가하세요:

  - name: Define image name and tag
    id: image_meta
    run: |
      OWNER_LOWERCASE=$(echo "${{ github.repository_owner }}" | tr '[:upper:]' '[:lower:]')
+     echo "owner_lowercase=${OWNER_LOWERCASE}" >> $GITHUB_OUTPUT
      IMAGE_TAG=$(date +'%Y%m%d-%H%M%S')
      echo "image_name=ghcr.io/${OWNER_LOWERCASE}/solid-connection-server" >> $GITHUB_OUTPUT
      echo "image_tag=${IMAGE_TAG}" >> $GITHUB_OUTPUT

---

98-98: ⚠️ 원격 서버의 GHCR 로그인에서도 소유자명 대소문자 불일치 (이전 리뷰의 미해결 항목)

Line 40-45의 GHCR 로그인 문제와 동일하게, 원격 서버에서의 docker login 명령어에서도 대소문자 불일치가 발생합니다.

문제:

• Line 93: export OWNER_LOWERCASE=$(echo "${{ github.repository_owner }}" | tr '[:upper:]' '[:lower:]')로 소문자 변수 생성
• Line 98: docker login ghcr.io -u ${{ github.repository_owner }} (원본 대소문자 사용)

변수가 정의되었으므로 일관성 있게 사용하세요.

수정:

  # 2. 서버가 GHCR에 로그인 (pull 받기 위해)
- echo "${{ secrets.GITHUB_TOKEN }}" | docker login ghcr.io -u ${{ github.repository_owner }} --password-stdin
+ echo "${{ secrets.GITHUB_TOKEN }}" | docker login ghcr.io -u ${OWNER_LOWERCASE} --password-stdin

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

.github/workflows/dev-cd.yml (3)

69-80: SCP 전송 시 에러 처리 추가 권장.

현재 설정 파일 전송 단계에서 scp 명령이 실패해도 워크플로우가 계속 진행됩니다. 예를 들어, 파일이 존재하지 않거나 네트워크 오류가 발생할 경우를 대비하세요.

다음과 같이 개선하면 실패 시 명확히 감지할 수 있습니다:

  - name: Copy config files to remote
    run: |
      echo "${{ secrets.DEV_PRIVATE_KEY }}" > deploy_key.pem
      chmod 600 deploy_key.pem
      
+     set -e  # 에러 발생 시 즉시 중단
+     
      scp -i deploy_key.pem \
        -o StrictHostKeyChecking=no \
        ./docker-compose.dev.yml \
        ./docs/infra-config/config.alloy \
        ./docs/infra-config/nginx.dev.conf \
        ${{ secrets.DEV_USERNAME }}@${{ secrets.DEV_HOST }}:/home/${{ secrets.DEV_USERNAME }}/solid-connection-dev/

---

118-119: docker compose 명령어 실행 시 환경 변수 설정 방식 정리.

현재 Line 119에서:

OWNER_LOWERCASE=$OWNER_LOWERCASE IMAGE_TAG=$IMAGE_TAG_ONLY docker compose -f docker-compose.dev.yml up -d

이 패턴은 작동하지만, OWNER_LOWERCASE가 Line 95에서 이미 export되었다면 불필요한 중복입니다. 일관성을 위해 다음 중 하나로 정리하세요:

옵션 1: 환경 변수를 명시적으로 선언하고 재사용

export OWNER_LOWERCASE=$(echo "${{ github.repository_owner }}" | tr '[:upper:]' '[:lower:]')
export IMAGE_TAG_ONLY=${{ steps.image_meta.outputs.image_tag }}
# ...
docker compose -f docker-compose.dev.yml up -d

옵션 2: 매번 인라인으로 설정 (현재 방식이지만 정리)

IMAGE_TAG=${{ steps.image_meta.outputs.image_tag }} \
OWNER_LOWERCASE=$(echo "${{ github.repository_owner }}" | tr '[:upper:]' '[:lower:]') \
docker compose -f docker-compose.dev.yml up -d

---

129-132: 오래된 이미지 정리 로직이 정상 작동하는지 검증 권장.

Line 129-132의 이미지 정리 로직은 수동으로 태그를 정렬하고 삭제합니다:

docker images "${IMAGE_NAME_BASE}" --format "{{.Tag}}" | \
sort -r | \
tail -n +6 | \
xargs -I {} docker rmi "${IMAGE_NAME_BASE}:{}" || true

주의할 점:

1. docker images 출력에 <none> 태그가 포함될 수 있으며, 이는 정렬 순서에 영향을 미칠 수 있습니다.
2. 태그가 6개 미만이면 아무것도 삭제되지 않습니다 (의도한 동작이라면 문제 없음).
3. || true로 인해 삭제 실패가 무시되므로, 실제 오류를 인지하기 어려울 수 있습니다.

개선 제안:

GHCR의 자동 정리 정책(Line 137-147, 최신 5개 유지)과의 일관성을 위해, 로컬 환경에서도 같은 개수를 유지하되, 더 명확한 로직을 고려하세요:

docker images "${IMAGE_NAME_BASE}" --format "table {{.Repository}}:{{.Tag}}\t{{.CreatedAt}}" | \
tail -n +2 | \
sort -k2 -r | \
tail -n +6 | \
awk '{print $1}' | \
xargs -r docker rmi

또는 snok/container-retention-policy 같은 로컬 도구도 검토해볼 가치가 있습니다.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: Hexeong
Repo: solid-connection/solid-connect-server PR: 552
File: .github/workflows/prod-cd.yml:40-45
Timestamp: 2025-11-10T01:55:03.812Z
Learning: GHCR (GitHub Container Registry)에서는 docker login 시 사용자명의 대소문자를 구분하지 않으며, 레지스트리 경로명은 항상 소문자로 정규화됩니다. 따라서 `github.repository_owner`를 그대로 사용해도 이미지 경로와의 대소문자 불일치 문제가 발생하지 않습니다.

.github/workflows/dev-cd.yml (1)

137-147: GHCR 이미지 정리 정책과 로컬 호스트 정리 정책의 일관성이 확인됩니다.

GHCR에서 최신 5개 태그를 유지(keep-n-tags: 5)하고, Line 131에서 로컬 호스트에서도 동일하게 최신 5개를 유지하는 방식으로 정책이 일관되어 있습니다. delete-untagged: true로 미태그 이미지도 정리하므로 레지스트리 저장소 효율이 좋습니다.

다만, 다음 사항을 확인해주세요:

• 정기적 정리 스케줄: 현재 구조에서는 성공적인 배포 시에만 정리가 실행됩니다(if: success()). 배포가 자주 이루어지지 않는 기간이 있다면 저장소가 누적될 수 있으니, 필요시 별도의 정기 정리 워크플로우(예: 주 1회 스케줄 실행)도 고려해보세요.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: Hexeong
Repo: solid-connection/solid-connect-server PR: 552
File: .github/workflows/prod-cd.yml:40-45
Timestamp: 2025-11-10T01:55:03.812Z
Learning: GHCR (GitHub Container Registry)에서는 docker login 시 사용자명의 대소문자를 구분하지 않으며, 레지스트리 경로명은 항상 소문자로 정규화됩니다. 따라서 `github.repository_owner`를 그대로 사용해도 이미지 경로와의 대소문자 불일치 문제가 발생하지 않습니다.